textview: add support for underline and strikethrough colors

This commit adds the GtkTextTag:underline-rgba and :strikethrough-rgba
properties and the necessary plumbing to apply these colors in GtkTextLayout.
With this change, you can alter the color of underlines including those
of type PANGO_UNDERLINE_ERROR.

You might want to alter the underline color to differentiate between
spelling and grammer mistakes. In code editors, it is convenient to
differentiate between errors and warnings.

Note that the GtkTextAppearance struct is public ABI and has no spare
room for new fields, so we are resorting to some tricky packing to store
the colors in the unused pixel field of the fg_color and bg_color structs.
This packing is accomplished by the macros in gtktextattributesprivate.h.

Signed-off-by: Christian Hergert <christian@hergert.me>
https://bugzilla.gnome.org/show_bug.cgi?id=402168

GtkRange: Fix scroll wheel direction for horizontal ranges

The expected behavior here is that scrolling up goes towards
larger values, ie to the right. This matches e.g. volume
sliders in gnome-shell.

https://bugzilla.gnome.org/show_bug.cgi?id=737175

scrolled window: Use shift to modify scroll direction

It has been a long-standing request to interpret scroll events
with Shift held down as horizontal instead of vertical, and
some applications are already doing this on their own.

https://bugzilla.gnome.org/show_bug.cgi?id=132197

HighContrast: Fix calendar selection

The selected day was not differentiated from other
days at all. Render it white on black instead.

https://bugzilla.gnome.org/show_bug.cgi?id=746369

HighContrast: Fix color chooser marks

The checkmark for the selected color was always black,
making it more or less invisible on dark colors.

https://bugzilla.gnome.org/show_bug.cgi?id=746368

HighContrast: catch backdrop:insensitive:active of GtkSwitch

https://bugzilla.gnome.org/show_bug.cgi?id=746358

HighContrast: unbreak GtkExpander

- the open state is checked, not active now

https://bugzilla.gnome.org/show_bug.cgi?id=746370

Updated Swedish translation

Updated Bulgarian translation

HightContrast: make tab buttons not shift size

- only 1px stroke for close buttons in notebook tabs

https://bugzilla.gnome.org/show_bug.cgi?id=746149

HighContrast: OSD fixes

- buttons
- labels
- scale trough/slider

https://bugzilla.gnome.org/show_bug.cgi?id=746148

Add to release notes

Mention that GtkSearchEntry now 'eats' some keys.

3.15.12

Updated Italian translation

Adwaita: desaturate selected color for backdrop

https://bugzilla.gnome.org/show_bug.cgi?id=746202

widget factory: Add an example for an insensitive model button

This will make it easier to make insensitive model buttons
appear properly in other themes.

wayland: Move gtk-shell compatibility version define too

https://bugzilla.gnome.org/show_bug.cgi?id=745721

Don't pass invalid rectangles to cairo

This causes pixman warnings, or worse.
Clearly, something is not quite right here, if we end up
redrawing tabs at a time when the allocation is set to (1, 1).
For now, avoid straining the error handling in the lower layers.

https://bugzilla.gnome.org/show_bug.cgi?id=746301

unrealize: Don't ref twice

After the recent change to add an outermost ref/unref pair,
we can do away with the inner one.

inspector: Remove flash timeout from widgets being unmapped

Otherwise the timeout can keep running, even if the widget is being
destroyed.

window: Protect against popover destruction on ::unmap

On popover_unmap(), perform the gtk_widget_unmap() call last, so the
GtkWindowPopover data is ensured to be alive throughout the function
if the popover widget is destroyed right on ::unmap.

https://bugzilla.gnome.org/show_bug.cgi?id=745829

widget: Take a ref during ::unrealize emission

In order to prevent invalid memory access/crashes if the widget is
destroyed in a callback of that signal.

https://bugzilla.gnome.org/show_bug.cgi?id=745829

widget: Take a ref during ::unmap emission

This ensures the widget data stays live long enough to perform invariants
check after emission if the widget happens to be destroyed on a callback.

https://bugzilla.gnome.org/show_bug.cgi?id=745829

Updated Bulgarian translation

reftests: Fix test on HighContrast theme

We are testing -gtk-icon-style and assume the theme doesn't touch it.
But HighContrast forces symbolic icons. And that breaks the reference
images.

So explicitly set "requested" for everything.

spinbutton: Fix get_text_area_size for vertical spinbuttons

Commit c5905b13b97012fc9e9424718bdce34e0989f512 changed the semantics of
this vfunc and get_entry_size but I only fixed one of them in the
spinbutton subclass.

https://bugzilla.gnome.org/show_bug.cgi?id=746282

wayland: Don't set the wl_surface user_data twice

wl_surface_add_listener already sets the user data pointer, so no need
to do it separately before.

https://bugzilla.gnome.org/show_bug.cgi?id=746141

wayland: Ignore setting the same cursor theme as was already set

If the name and size of the theme is identical to the current
configuration, do nothing.

https://bugzilla.gnome.org/show_bug.cgi?id=746141

wayland: Support scaling of theme based cursors

Support scaling of cursors created from themes. The default scale is
always 1, but if the pointer cursor surface enters an output with a
higher scale, load the larger version of the cursor theme and use the
image from that theme.

This assumes the theme size is set to one that fits with an output scale
= 1.

https://bugzilla.gnome.org/show_bug.cgi?id=746141

wayland: Put interface version defines in a common place

So far only one, but put it somewhere all files can see it.

https://bugzilla.gnome.org/show_bug.cgi?id=746141

wayland: Don't set cursor surface scale when we don't have a surface

The setting of the the surface scale even when the surface is not
created from a surface was introduced due to a crash when getting the
buffers when dividing by the scale. The only reason I can see this is
that we get the buffer from a non-existing surface when the wl_cursor
has not yet been set.

Instead, use the name field to avoid trying to use the non-existing
surface, effectively avoiding the division-by-zero that way.

https://bugzilla.gnome.org/show_bug.cgi?id=746141

HC: no border for insensitive backdrop buttons

https://bugzilla.gnome.org/show_bug.cgi?id=746192

Updated Vietnamese translation

Signed-off-by: Trần Ngọc Quân <vnwildman@gmail.com>

widget-factory: Add an insensitive toolbutton

This will help getting their theming right in HighContrast.

widget-factory: Add tab close buttons

This will make it easier to fix issues with such buttons
in HighContrast.

gtk3-demo: Stop using follow-state

It is no longer needed.

print dialog: Stop using follow-state

It is no longer needed.

app chooser widget: Stop using follow-state

It is no longer needed.

places sidebar: Stop using follow-state

It is no longer needed.

widget-factory: Stop using follow-state

The property is no longer needed, cell renderers now
always follow state.

Updated Swedish translation

Updated Slovenian translation

Updated Danish translation

gemfile: needs add source rubygems.org

`bundle install` command on fedora 21 was failing
with following message:

Your Gemfile has no gem server sources. If you need
gems that are not already on your machine, add a line
like this to your Gemfile:
source 'https://rubygems.org'

After adding that line it worked again.

https://bugzilla.gnome.org/show_bug.cgi?id=746205

Updated POTFILES.in and POTFILES.skip

entry: Make code do what it's meant to do

With the 3.0 transition, this code went from just querying the entry's
height request to doing a full size request.
Then it got code to revert the features that a full size request does.
And then it grew code that manually computed the baseline.

Avoid this and just do what happened back in the days: Do a regular
height request.

This changes the semantics of the get_frame_size() vfunc wrt its
behavior towards subclasses overwriting the get_height() vfuncs, but I'm
happy to live with that.

cellrendererpixbuf: Always follow state

Applications forget to set the follow-state property and then some
themes used a style where normal symbolic icons have the same color as
prelit or selected backgrounds and that suddenly made icons invisible
and who wants that?

https://bugzilla.gnome.org/show_bug.cgi?id=746201

gtkstacksidebar: fix missing macro

commit e41cadd835f9c2 forgot to change some macros.

Updated Bosnian translation

More documentation fixes

With this commit, gtk3-unused.txt is empty (for the first
time in a looong time).

GtkStackSidebar: Fix incomplete renaming

Some of the boilerplate macros were missing. Oops

More documentation fixes

Docs: Update ignored header list

This reduces the warning spam somewhat.

Documentation fixes

Add a few missing symbols.

Fix a typo in the docs

gtk-doc complains about parameter name mismatches.

Translation pa updated for Gnome

updated Danish translation

updated Danish translation

menu button: Improve accessibility

Rework the way we assign an accessible name to menu buttons,
to make sure we pick up a label, should the button contain
one, and only override the name with "Menu" as a fallback.

testsuite: Set some environment variables

These were set manually in some tests, but not consistently.

testsuite: Drop test bus

It seems to be buggy in ways that make the test fail
with a critical when the test bus is brought down.
At the same time, drop manual settings of environment
variables that we can set globally.

Updated Bosnian translation

Updated Bosnian translation

Update Catalan translation

3.15.11

Fix make dist

Add a missing quote.

Drop unused files

Updated Slovak translation

Updated Italian translation

Update Catalan translation

Revert "iconhelper: Don't check states"

This reverts commit 63f59dde3a2740057fb24115b58100af630aa453.

It turns out, the state was not just necessary for style computation,
but also for tracking RTL and LTR. And so it broke the reftests.

Add some reftests related to textview tags

This test shows that a) tags override the text view style and
b) tags override each other in the expected way.

iconhelper: Use the screen's icon theme

... and not the default one.

iconhelper: Replace function

The old one just calls the other one.

iconhelper: Move function from style context into only user

iconhelper: Don't check states

This is the wrong thing to do in today's world.

App Chooser: Fix app icons in HighContrast

We need to follow-state here as well, to make sure the
icons are visible on selected rows.

Updated Korean translation

gtkentry: Document vfuncs in GtkEntryClass

Clarify that the signal handler vfuncs can be NULL, nothing else can,
and that they all have default implementations.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtktreeselection: Fix an abort on selecting an invalid range

gtk_tree_selection_real_modify_range() has a g_return_if_fail() if the
start or end paths passed to it do not correspond to real tree nodes.
However, GtkTreePaths inherently do not have to be valid, so it should
be acceptable to call gtk_tree_selection_select_range() with
non-existent paths. Replace the g_return_if_fail() by a silent return,
and add a unit test.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtknotebook: Add more non-NULL checks for cur_page

These were not spotted by scan-build, but from some brief mental
reasoning could potentially be problem areas.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

Gestures: Properly handle weak references

Every call to g_object_add_weak_pointer must be matched with
a g_object_remove_weak_pointer call, or we will get invalid
writes inside GObject.

https://bugzilla.gnome.org/show_bug.cgi?id=746013

wayland: Only bind to supported gtk-shell versions

The gtk-shell Wayland protocol extension is not meant to be backward
compatible right now, so avoid binding to any version that is not the
one supported.

https://bugzilla.gnome.org/show_bug.cgi?id=745721

build: Fix generating introspection on MacOS

We need to filter out platform-specific sources from the files that we
use with the introspection scanner.

In this specific case I could have moved the gtkclipboard-quartz.c and
the gtkdnd-quartz.c files out of the $(gtk_base_c_sources) variable, but
doing a filter-out on the variable itself is more resilient in case we
eventually add files and we forget about the result.

inspector: Add some margin to css editor

Requested in
https://bugzilla.gnome.org/show_bug.cgi?id=745918

window: reset both press/drag gestures when move/resize drag starts

Now that this is split in two separate gestures, both must be reset
when the WM grabs the pointer. Also, do on resize drags like on move
drags, and claim the gesture before resetting, so the ownership is
properly transferred across any other widgets.

https://bugzilla.gnome.org/show_bug.cgi?id=745969

Updated Swedish translation

Updated Swedish translation

notebook: Fix boolean check

Commit c352093bdeb2be5390607de13eaf7a21e93272ef swapped the condition
when turning it into a return_if_fail().

entrycompletion: Ensure transient_for is set before showing window

gtk_entry_set_completion() (and _gtk_entry_completion_connect underneath)
may happen when the entry has no toplevel yet. This window acts as the
"parent" for subsurface windows on wayland, and is indispensable to create
its backing wl_subsurface, so this resulted in non-visible popups there.
Fix this by ensuring transient_for is set before showing the popup.

https://bugzilla.gnome.org/show_bug.cgi?id=745463

gtknotebook: Fix potential use of uninitialised variables

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkprintunixdialog: Fix a potential g_object_unref(NULL) call

The code above checks whether (printer == NULL), so we’d better do so
here as well.

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

Revert "csd: Drop the GTK_FRAME_EXTENTS requirement"

This reverts commit fb9a6bb6d8d6b60b25c9b9853decbcea938f7863.

In a recent test, I've found that Xfce and Mate now support
this, so they will not be affected by this requirement. And
adding the check back will solve the 'client-side shadow'
problem in KDE.

stylecontext: Store the right style in the cache

Store the newly computed style, not the old and wrong one.

Fixes HighContrast menuitem color sometimes being black-on-black for
selected items.

gtkclipboard: Fix a potential g_signal_handler_disconnect(NULL) call

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkprintcontext: Fix several potential g_object_[un]ref(NULL) calls

The page_setup of a GtkPrintContext or GtkPrintUnixDialog is nullable,
so all reference count changes to it have to be guarded against NULL
values.

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtktreeselection: Fix a potential use of uninitialised variables

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtktreeselection: Fix potential NULL pointer dereferences

_gtk_rbtree_first() can potentially return NULL if the RB tree is empty,
which would result in NULL pointer dereferences in the GtkTreeSelection
code. Gracefully handle them.

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtktextview: Fix a definite use of an uninitialised variable

This seems to have been a typo in the original code, and allowed access
to virtual_cursor_y when it was uninitialised.

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtknotebook: Remove dead variable assignments

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760